Skip to content

[Website] Renaming stored Playgrounds #2486

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

adamziel
Copy link
Collaborator

@adamziel adamziel commented Aug 8, 2025

Motivation for the change, related issues

A limited version of #2124 that only enables renaming stored Playgrounds. It does not enforce choosing a name when the Playground is first saved. This is to make the review and debugging easier – the original PR is larger and breaks the E2E tests.

Testing Instructions (or ideally a Blueprint)

This PR ships an E2E test.

For manual testing:

  • Checkout this branch
  • Start the dev server npm run dev
  • Go to the Playground manager
  • Click save on a temporary site
  • Click on the kebab menu of that site and click Rename
  • Rename the site and confirm the changes are saved
  • Refresh the page and confirm the new name is still there

@adamziel
Copy link
Collaborator Author

adamziel commented Aug 9, 2025

The only failing test is the one shipped with this pr - good. No tricky failures related to switching between playgrounds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant